Update to python3 and elastalert v0.2.4#174
Closed
Karql wants to merge 1 commit intobitsensor:masterfrom
Closed
Conversation
|
Are this is not need any more? it solved the bug with jira 1.0.15 plugin and there are some requirements as i remember |
Author
|
@hottwister Also in this issue YelpArchive/elastalert#1654 they wrote that it works ;) |
Closed
|
@balthazar is rite. I used the following Dockerfile is my environment. I didn't go uptown 0.2.4 since I only needed some features in 0.2.2. |
Is this after running |
|
docker build error |
Karql
force-pushed
the
update-elastalert
branch
from
9c7936d to
ea34df2
Compare
Author
|
The problem is with new cryptography lib (pyca/cryptography#5771) Try now. |
|
I have confirmed that the problem has been resolved. |
|
Cryptography seems to be okay up to 3.3. The issue says alpine: 3.13 and installs rust and cargo. The docker image of node seems to be up to alpine: 3.12 so far, so I understood that it is correct to avoid the problem by installing the version before 3.0 or the version before 3.4 like this time. |
Babel Upgrade (6 → 7) index.js require('babel-register'); → require('@babel/register'); package.json Library Update src/common/config/server_config.js
before after src/common/file_system.js
before after src/controllers/rules/index.js
before after src/controllers/templates/index.js
before after src/handlers/test/post.js before after Makefile before after |
Author
|
There was a resone for using 3.11 version - don't remember why now. cryptography 2.9.2 was the latest version when i've made changes to Dockerfile so now I stick to this version.
You can always made your own version and use the latest stuff if you want. |
The latest cryptography is 3.4.4.
|
|
I found that the following method works.
Dockerfile |
Author
Means when I created this PR Ok I remembered - python version was other problem: #122 (comment) alpine:3.11 was also the latest version then. I checked now:
It is build successfuly but don't try does it works correctly. |
I just confirmed that the docker image was created successfully. alpine: 3.11・・・rust 1.39.0-r0 alpine: By setting 3.13, I was able to clear the condition of rust> = 1.45, so I found that no error occurred. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update elastalert to v0.2.4
I've tried to make as few changes as possible.